-
Notifications
You must be signed in to change notification settings - Fork 48
Brody/fix install blocks #381
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
beklapko
commented
Sep 4, 2025
- This fixes/adds missing install code tabs for some JS integration pages.
- Removes a callout from several docs (the callout linked to a page that's in the old docs but not the new docs, and didn't seem critical to me to have right now).
Preview ID generated: preview-brodyf-1757023002-0b008f4 |
Preview ID generated: preview-brodyf-1757023098-c97c0e5 |
This can probably wait to go through until next week. I can probably just rerun the script on main next week to avoid merge conflicts that get introduced between now and then. If it looks good though and a brave soul wants to merge, go for it! |
@langchain/anthropic @langchain/core | ||
</Npm2Yarn> | ||
```bash yarn | ||
yarn add @langchain/anthropic @langchain/core |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might be able to create a re-usable component for this to save some typing and make sure that all package managers are always included.
<jsInstalls packages=["@langchain/anthropic", "@langchain/core"]/>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(just an idea, not meant to be blocking since its very low priority)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's definitely the more appropriate/long term fix. This is kind of a patch/bandaid until we can do that.
Replacing the changes in this PR with this one: #415 |